Skip to content

(CODUX-216) Wire agent attachments into the canonical Nodes workspace - #2976

Merged
numnx merged 1 commit into
feature/CODUX-216-nodes-full-integrationfrom
task/feature-codux-21-t23-codex-2a84f564-mrhncxn0
Jul 12, 2026
Merged

(CODUX-216) Wire agent attachments into the canonical Nodes workspace#2976
numnx merged 1 commit into
feature/CODUX-216-nodes-full-integrationfrom
task/feature-codux-21-t23-codex-2a84f564-mrhncxn0

Conversation

@numnx

@numnx numnx commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

🧩 Task: Wire agent attachments into the canonical Nodes workspace

Automated task PR opened by Code UX for T23.

📋 Summary

Sprint: Sprint 216 · Nodes Full Integration · Task: T23

Turn the existing project-scoped node-flow foundation and browser-local canvas into a canonical, secure, versioned automation control plane with registry-driven nodes, encrypted credentials, durable execution, governed integrations, isolated custom nodes, agent authoring, dashboard integration, and authenticated headless readiness.

🤖 Model & Provider

Provider Codex CLI
Model gpt-5.6-sol

⏱️ Timing

Started 2026-07-12 10:24:50 UTC
Finished 2026-07-12 10:34:05 UTC
Duration 9m 14s

📊 Token Usage

Metric Value
Input tokens 155,834
Cached input tokens 4,180,224
Output tokens 19,063
Total tokens 4,355,121
Invocations 1
Tool calls 46
Active time 8m 44s

Included usage estimate (subscription/local login): $3.44

✅ QA Review

⏳ QA review runs after this PR is opened. Check the task card in the dashboard for the outcome once it lands.

📝 Full Task Prompt
## Objective
Complete the dashboard agent attachment integration required by T17 so the canonical project-scoped Nodes workspace can manage governed flow attachments.

## Scope
- `dashboard/src/v2/NodesPage.tsx`
- `dashboard/src/v2/lib/agent-preset-api.ts`
- `dashboard/src/v2/lib/node-flow-api.ts`
- `dashboard/src/v2/components/nodes/NodeFlowInspector.tsx` only if required for integration
- Dashboard tests under `tests/dashboard/v2/` and related dashboard library tests

## Requirements
1. When a project is selected, load its agent presets through the existing project-agent API. When a flow is selected, load that flow's attachments through the existing node-flow attachment API.
2. Maintain attachment state, selected agent state, loading state, mutation state, and user-visible error handling. Abort or ignore stale requests when the selected project or flow changes.
3. Wire attach to `attachNodeFlowToAgent` and detach to `detachNodeFlowFromAgent`. Refresh the selected flow's attachments after each successful mutation, clear or preserve selection deterministically, and keep project ownership enforced by the backend APIs.
4. Pass the actual agents, attachments, selected agent id, mutation state, and attach/detach callbacks into `NodeFlowInspector`; remove the hard-coded empty arrays and no-op callbacks currently used by `NodesPage`.
5. Preserve metadata-only behavior: attachment UI may show agent and skill metadata but must never render credentials, decrypted values, or custom source.
6. Preserve accessible loading, error, empty, project-switching, and keyboard interaction states.

## Verification
- Add tests covering agent preset loading, existing attachment loading, attach, detach, project switching, failed API calls, loading/error recovery, and keyboard accessibility.
- Run `pnpm run typecheck:dashboard`.
- Run focused Nodes dashboard tests and the full dashboard suite, using single-worker execution if required by the validation environment.
- Run `pnpm run build` and `pnpm run test:e2e:credentialed-automation`.

## Constraints
- Reuse the existing dashboard fetch helpers and governed backend contracts.
- Do not add browser-local persistence for attachments.
- Do not weaken project ownership or attached-flow capability authorization.
🌿 Branch Info

Base: feature/CODUX-216-nodes-full-integration
Head: task/feature-codux-21-t23-codex-2a84f564-mrhncxn0


🤖 Generated by Code UX

@numnx
numnx merged commit c4e2e0d into feature/CODUX-216-nodes-full-integration Jul 12, 2026
20 checks passed
@numnx
numnx deleted the task/feature-codux-21-t23-codex-2a84f564-mrhncxn0 branch July 12, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant